-
Notifications
You must be signed in to change notification settings - Fork 2.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove ReCaptchaMode and DetectionThreshold #17229
Conversation
@rjpowers10 I have to update docs and release notes. But, can you please test this out? The expected behavior is that if the reCapcha feature is enabled, then will always present the challenge. We no longer have threshold. If the user tried multiple times their account will get locked after 5 tries by default during login. I think if we ever use v3 of Google, then the challenge is handled differently but either way we won't need to worry about doing threshold ourselves. |
@rjpowers10 were you able to test this out? |
@MikeAlhayek I took a quick look and the ReCaptcha displayed in all four locations as expected.
|
Thanks for confirming. |
Fix #17200